feat(docs): theme 2.0.0, the lighter blue accent, and a light/dark logo pair - #8
Merged
Conversation
…go pair Three things the landing surfaced when di joined it as the fifth panel (btravstack/btravstack.github.io#43). Closes #6. Closes #7. **The accent.** `--accent` was `#2A62B8`, the logo's deep blue. The landing paints each package name in the raw accent against a `#100F12` card, and that hex measures 3.22 there — the other four packages sit at 5.00–7.25. It is now `#3E7FD4`, the logo's *lighter* blue: 4.72 on the dark card, 7.10 as darkened text on white. The original reasoning is untouched and still holds — blue is plumbing, and the hex is one the mark itself already paints (barrel rim, plunger, needle hub), so chrome and artwork stay one color. **The logo pair.** di shipped only `logo.svg`, and it was the one package without a `logo-{light,dark}` pair. Three of its values read against the canvas rather than against the artwork, so on a light background the near-white barrel washed out and the mark read as a floating beet with a blue cap. `logo-dark` is the existing file unchanged; `logo-light` inverts exactly those three — barrel `#EAF2FB`→`#2A2730`, plunger `#2A62B8`→`#3E7FD4`, needle `#9FB2C8`→`#5A6675`. The ticks stay at 50% opacity: 1.89 against the dark barrel versus 1.81 against the light one, so they read equally faint either way. The hero and the nav logo now take the pair; `logo.svg` stays the favicon and the JSON-LD mark. **The theme.** Catalog moves 1.7.0 → 2.0.0, which is where `--pkg-di` lives. The major is the removal of `--pkg-demesne` / `--pkg-start`; neither is referenced here, so nothing breaks. pnpm recorded the new version as a pinned `minimumReleaseAgeExclude` entry — replaced with the unversioned name and the rationale comment the other repos use, so the next release needs no entry. Docs build clean: `--accent: #3E7FD4` in the emitted CSS, no `2a62b8` left, all five `--pkg-*` tokens present and both retired ones gone, and both logo variants verified in both themes.
There was a problem hiding this comment.
Pull request overview
Updates the di documentation site’s visual identity to align with @btravstack/theme@2.0.0, switching the package accent to the logo’s lighter blue and introducing theme-aware logo assets for light/dark canvases.
Changes:
- Bump
@btravstack/themefrom1.7.0→2.0.0and record a workspace policy exception for adopting that first-party config immediately. - Repoint the docs accent token from
#2a62b8to#3E7FD4(the logo’s lighter blue). - Add
logo-light.svg/logo-dark.svgand wire them into the VitePress home hero and navbar logo.
Reviewed changes
Copilot reviewed 4 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| pnpm-workspace.yaml | Bumps @btravstack/theme to 2.0.0 and adds minimumReleaseAgeExclude entry for @btravstack/theme. |
| pnpm-lock.yaml | Updates lockfile to reflect @btravstack/theme@2.0.0. |
| docs/public/logo-light.svg | Adds light-canvas logo variant with inverted canvas-dependent fills. |
| docs/public/logo-dark.svg | Adds dark-canvas logo variant for theme-aware swapping. |
| docs/index.md | Switches hero image to use light/dark logo variants. |
| docs/.vitepress/theme/custom.css | Updates --accent to #3E7FD4 and expands rationale comment. |
| docs/.vitepress/config.ts | Switches navbar logo config to light/dark variants. |
Files not reviewed (1)
- pnpm-lock.yaml: Generated file
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
`pnpm format --check` failed CI on this file. oxfmt normalises CSS hex values
to lowercase, which is why every sibling site's `--accent` is lowercase and why
this file's own `#2a62b8` was before it changed. The comment's hex follows, so
the file does not mix cases with its own declaration.
Note this is the opposite of the markdown convention: prose in changesets and
`design.md` writes hexes uppercase. Each file type is internally consistent.
Root cause of the miss: the shared pre-commit hook in @btravstack/lefthook
globs `*.{ts,tsx,js,jsx,json,yaml,yml,md}`, which has no `css`, so this file was
never formatted locally — the hook reported three files (config.ts, index.md,
pnpm-workspace.yaml) and skipped this one. CI runs `oxfmt .` over everything.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #6. Closes #7. Companion to btravstack/btravstack.github.io#43, which added di to the landing as the fifth panel and surfaced all three of these.
The accent —
#2A62B8→#3E7FD4The original reasoning in
custom.cssstands and is kept: blue is plumbing, and the hex should be one the logo itself paints so chrome and artwork stay one color. Only the specific blue changes.The landing paints each package name in the raw accent against a
#100F12card. Measured there:#3E7FD4#2A62B8(what this file had)--pkg-temporal#6B76F2--pkg-unthrown#3FB0A5#2A62B8is the logo's deep blue and lands well below the 5.00–7.25 the other four occupy.#3E7FD4is the logo's lighter blue — the barrel rim, the plunger, the needle hub — so it is still a hex the mark already uses. It is--pkg-diin the theme, so this site and the landing panel now agree.The logo pair
di was the only package shipping
logo.svgalone. Three of its values read against the canvas rather than against the artwork:#EAF2FB#2A2730#2A62B8#3E7FD4#9FB2C8#5A6675Left alone on a light background the near-white barrel washes out and the mark reads as a floating beet with a blue cap.
logo-dark.svgis the existing file byte-for-byte — it was drawn for a near-black canvas.logo-light.svginverts exactly those three and nothing else: the deep blue thumb rest and rod, the rim, the beet and the drop are untouched. The graduation ticks stay#3E7FD4at 50%, landing at 1.89 against the dark barrel versus 1.81 against the light one, so they read equally faint either way.Wired into the hero (
index.md) and the nav logo (config.ts), matching entity.logo.svgremains the favicon and the JSON-LD organization mark.The theme — 1.7.0 → 2.0.0
Where
--pkg-dilives. The major is the removal of--pkg-demesne/--pkg-start; grepping this repo returns zero references to either, so nothing breaks.One incidental:
pnpm installrecorded the new version as a pinnedminimumReleaseAgeExcludeentry ('@btravstack/theme@2.0.0'). I replaced it with the unversioned name plus the rationale commententityandunthrownboth use, so the next theme release needs no new entry. Only@btravstack/themeis listed — the other first-party@btravstack/*catalog entries can join it when they next bite, rather than changing policy for five packages in a docs PR.No changeset:
@btravstack/diitself is unchanged, and@btravstack/di-docsis private.Verified
pnpm --filter @btravstack/di-docs buildclean. In the emitted CSS:--accent: #3E7FD4, no2a62b8anywhere, all five--pkg-*tokens present and both retired tokens gone. Both logo variants screenshotted in both themes — hero and nav swap correctly.